Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Rust]mainとのconflict解消 #204

Merged

Conversation

qwerty2501
Copy link
Contributor

内容

T/O

関連 Issue

refs #128

qwerty2501 and others added 14 commits May 9, 2022 02:58
本当はデフォルト引数を消したかったが、使ってる人がいる可能性があるためC++で使ってる場合はそのままにするように修正した
* load_model関数を切り出す

* load_modelとis_model_loadedを足した

* 使われてないエラーメッセージを消去

* Update core/src/core.cpp

Co-authored-by: qwerty2501 <[email protected]>

* Update core/src/core.cpp

Co-authored-by: qwerty2501 <[email protected]>

* Update core/src/core.cpp

Co-authored-by: qwerty2501 <[email protected]>

* pythonのラッパーの型を変更

* load_all_models追加

* return true

Co-authored-by: qwerty2501 <[email protected]>
* Catch2をプロジェクト導入して参考となる1ケースを追加した

* リリース用ビルドにはテストビルドは不要のため無効化する変数を追加した

* テストコードの短縮化を行った

* READMEにテスト実行方法を追加

* テストをビルドするかどうかのフラグをデフォルトOFFにした

* テストをビルドするかどうかのフラグを変更したことによりREADMEを修正

* MSVCに対して足りないオプションを追加

- utf8文字列でコンパイルするように指定
- C++20を明示的に指定
- __cplusplusの値を利用中のC++のバージョンに合わせるように指定(ないとC++98相当になるとか)

* Configがリリースの場合のみに最適化オプションを指定するように修正

* オプションの指定を短くまとめた

* coreの標準ライブラリもバージョンアップした

* compile optionsの修正

* Catch2にもCXX_STANDARD 20を追加

* Windows環境でビルドできるように設定を修正

VOICEVOX#121 (comment)

* format 効いてしまっていたところを修正
* .gitignore open_jtalk_dic in example

* modify: example/python/.gitignore
* C++サンプルコードへのリンクを追加

* #readme

* Update README.md
* コード署名

* build_util

* artifact/

* a

* remove
* inputのbooleanは文字列として渡ってくるので判定を修正

* 修正もれ
@qwerty2501 qwerty2501 marked this pull request as draft July 23, 2022 17:31
@qwerty2501
Copy link
Contributor Author

#202 READMEの修正をmergeしてからreview開始すること

@qwerty2501 qwerty2501 force-pushed the feature/resolve-conflict-from-main branch from 605a831 to c2c1d12 Compare July 23, 2022 17:34
@qwerty2501 qwerty2501 changed the title mainとのconflict解消 [Rust]mainとのconflict解消 Jul 23, 2022
@qwerty2501 qwerty2501 marked this pull request as ready for review July 24, 2022 04:43
@qwerty2501
Copy link
Contributor Author

@Hiroshiba @PickledChair reviewできそうになったのでお願いします

Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!!

Copy link
Member

@PickledChair PickledChair left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

手元でも conflict が解消されていることを確認しました。LGTM!

@PickledChair
Copy link
Member

こちらマージします!

@PickledChair PickledChair merged commit db28cbf into VOICEVOX:rust Jul 24, 2022
@qwerty2501 qwerty2501 mentioned this pull request Jul 24, 2022
qwerty2501 added a commit to qwerty2501/voicevox_core that referenced this pull request Jul 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants